Add sz build_runner build command
#997
Conversation
| argParser.addFlag( | ||
| 'delete-conflicting-outputs', | ||
| help: | ||
| 'By default, the user will be prompted to delete any files which already exist but were not known to be generated by this specific build script.', |
There was a problem hiding this comment.
Not sure if promoting the options works with our CLI. Maybe we should always use the --delete-conflicting-outputs flag because I think this is the option to always use.
There was a problem hiding this comment.
promoting the options works with our CLI
I don't know what do you mean exactly. Does that mean that you are unsure if we as developers would notice having that flag for the command?
I'm okay with defaulting it with true, since like you said that's the option we're always using anyways.
There was a problem hiding this comment.
Normally, when there is a conflict, the CLI prompts three options. You can then press key 1, 2 or 3.
[INFO] Found 25 declared outputs which already exist on disk. This is likely because the`.dart_tool/build` folder was deleted, or you are submitting generated files to your source repository.
Delete these files?
1 - Delete
2 - Cancel build
3 - List conflicts
However, because we don't show the prints from the Dart CLI, you will not see this message and you can't press any of these options.
Therefore, I would also hard-code --delete-conflicting-outputs to true because as a Sharezone Repo CLI user can't do anything else and wouldn't make sense to set --delete-conflicting-outputs to false.
| argParser.addFlag( | ||
| 'delete-conflicting-outputs', | ||
| help: | ||
| 'By default, the user will be prompted to delete any files which already exist but were not known to be generated by this specific build script.', |
There was a problem hiding this comment.
promoting the options works with our CLI
I don't know what do you mean exactly. Does that mean that you are unsure if we as developers would notice having that flag for the command?
I'm okay with defaulting it with true, since like you said that's the option we're always using anyways.
Needed for #963
Usage: